Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Release/315.0.0 #5415

Merged
merged 4 commits into from
Feb 28, 2025
Merged

Release/315.0.0 #5415

merged 4 commits into from
Feb 28, 2025

Conversation

infiniteflower
Copy link
Contributor

@infiniteflower infiniteflower commented Feb 28, 2025 •
edited
Loading

Explanation

This PR changes the BridgeController and BridgeStatusController state object to contain all the relevant fields at the root object, rather than keeping them behind a redundant field.

References

Changelog

@metamask/bridge-controller

  • BREAKING: Change BridgeController state structure to have all fields at root of state
  • BREAKING: Change BridgeController state defaults to null from undefined
  • ADDED: Mobile feature flags

@metamask/bridge-status-controller

  • BREAKING: Change BridgeStatusController state structure to have all fields at root of state
  • BREAKING: Redundant type BridgeStatusState removed from exports

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've highlighted breaking changes using the "BREAKING" category above as appropriate
  • I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

@socket-security Socket Security
Copy link

socket-security bot commented Feb 28, 2025 •
edited
Loading

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: npm/@metamask/bridge-controller@1.0.0

View full report↗︎

Have feedback? Participate in our User Experience Survey 📊

@infiniteflower infiniteflower marked this pull request as ready for review February 28, 2025 16:11
@infiniteflower infiniteflower requested review from a team as code owners February 28, 2025 16:11
@infiniteflower infiniteflower merged commit 15e6a7e into main Feb 28, 2025
138 checks passed
@infiniteflower infiniteflower deleted the release/315.0.0 branch February 28, 2025 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amitabh94 amitabh94 amitabh94 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@infiniteflower @amitabh94